Skip to content

ci: add trial CodeQL workflow - #10344

Open
drousselbot wants to merge 1 commit into
NousResearch:mainfrom
drousselbot:ci/codeql-trial-clean
Open

drousselbot wants to merge 1 commit into
NousResearch:mainfrom
drousselbot:ci/codeql-trial-clean

Conversation

@drousselbot

Copy link
Copy Markdown

Summary

  • add a trial CodeQL workflow for Python and JavaScript/TypeScript
  • pin CodeQL and checkout actions to exact SHAs
  • validate CodeQL on a PR before deciding whether to require it or proceed with Renovate

Validation

  • YAML parsed locally
  • action refs verified as SHA-pinned

Notes

  • This is intentionally a trial rollout per our current workflow discussion.
  • Renovate should wait until CodeQL is confirmed working cleanly.

@alt-glitch alt-glitch added type/security Security vulnerability or hardening P3 Low — cosmetic, nice to have labels Apr 26, 2026

@teknium1 teknium1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding a narrowly scoped, SHA-pinned CodeQL trial. The current main workflow tree has no CodeQL workflow; this addition is therefore not superseded.

Problems

  • .github/workflows/codeql.yml:31 pins actions/checkout to v4, while current workflows use the v6.0.2 pin, including .github/workflows/ci.yml:48. Please align this new workflow with the repository-wide pin.

Suggested changes

  • Use actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 at .github/workflows/codeql.yml:31.

Automated hermes-sweeper review.

matrix:
language:
- python
- javascript-typescript

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please align this with the repository-wide checkout pin: current workflows, including .github/workflows/ci.yml:48, use de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2 rather than the older v4 reference.

@teknium1 teknium1 added sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation sweeper:risk-security-boundary Sweeper risk: may affect sandboxing, auth, credentials, or sensitive data sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jul 12, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation sweeper:risk-security-boundary Sweeper risk: may affect sandboxing, auth, credentials, or sensitive data type/security Security vulnerability or hardening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants